Skip to content

Fix: Apprise test button shown without saving #2627

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
alexjustesen merged 2 commits into from
Jan 3, 2026
Merged

Fix: Apprise test button shown without saving #2627

alexjustesen merged 2 commits into from
Jan 3, 2026

Conversation

@svenvg93
Copy link
Contributor

@svenvg93 svenvg93 commented Jan 3, 2026

📃 Description

This PR fixes the Apprise test button being shown without saving the settings. Now the button is only shown after saving the settings.

As the used AppriseChannel uses the saved settings I choose to only show the button after saving, instead of redoing the whole logic.

📖 Documentation

🪵 Changelog

➕ Added

  • Text to indicate the button will appear after saving.

🔧 Fixed

@svenvg93 svenvg93 requested a review from alexjustesen as a code owner January 3, 2026 10:09
@svenvg93 svenvg93 changed the title Bug: Fix Apprise test button shown without saving Fix: Apprise test button shown without saving Jan 3, 2026
@alexjustesen alexjustesen merged commit 07aaf84 into alexjustesen:main Jan 3, 2026
9 checks passed
@svenvg93 svenvg93 deleted the 2618-fix-test-button branch January 3, 2026 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexjustesen alexjustesen alexjustesen approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug] Apprise test notification throws error when not saved

2 participants

@svenvg93 @alexjustesen